Skip to content

Conversation

@martincostello
Copy link
Owner

Update to preview 6 of ASP.NET Core 10.

Update to preview 6 of ASP.NET Core 10.
Copilot AI review requested due to automatic review settings July 15, 2025 16:00
@martincostello martincostello added dependencies Pull requests that update a dependency file .NET Pull requests that update .NET code labels Jul 15, 2025
@martincostello martincostello enabled auto-merge (rebase) July 15, 2025 16:00
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the project to ASP.NET Core 10 Preview 6, bumping package and SDK versions, consolidating OpenAPI usings, and adjusting collection initializers to match updated APIs.

  • Bump all ASP.NET Core, EF Core, OpenAPI, and Swashbuckle dependencies to Preview 6 (and stable 2.0 for Microsoft.OpenApi)
  • Simplify using directives by replacing Microsoft.OpenApi.Models (and related namespaces) with the new Microsoft.OpenApi umbrella
  • Replace shorthand [] initializers with explicit collection constructors where needed

Reviewed Changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tests/TodoApp.Tests/TodoApp.Tests.csproj Updated test dependencies to ASP.NET Core 10 Preview 6
tests/TodoApp.Tests/OpenApiTests.cs Consolidated OpenAPI usings to Microsoft.OpenApi
tests/TodoApp.Tests/OpenApiTests.Schema_Is_Correct_schemaUrl=swagger.verified.txt Updated test snapshot formatting/order for new OpenAPI output
tests/TodoApp.Tests/OpenApiTests.Schema_Is_Correct_schemaUrl=openapi.verified.txt Same snapshot updates for OpenAPI schema
src/TodoApp/TodoApp.csproj Bumped app dependencies to ASP.NET Core 10 Preview 6 and stable OpenAPI
src/TodoApp/OpenApi/Swashbuckle/SwashbuckleOpenApiEndpoints.cs Updated OpenAPI usings
src/TodoApp/OpenApi/Swashbuckle/ExampleFilter.cs Updated OpenAPI usings
src/TodoApp/OpenApi/Swashbuckle/AddServersFilter.cs Updated OpenAPI usings
src/TodoApp/OpenApi/Swashbuckle/AddDocumentTagsFilter.cs Updated OpenAPI usings and collection initializer
src/TodoApp/OpenApi/ExamplesProcessor.cs Updated OpenAPI usings
src/TodoApp/OpenApi/AspNetCore/AspNetCoreOpenApiEndpoints.cs Updated OpenAPI usings and security schemes initializer
src/TodoApp/OpenApi/AspNetCore/AddSchemaDescriptionsTransformer.cs Updated OpenAPI usings
src/TodoApp/OpenApi/AspNetCore/AddExamplesTransformer.cs Updated OpenAPI usings
global.json Updated .NET SDK version to 10.0.100-preview.6
Comments suppressed due to low confidence (2)

src/TodoApp/OpenApi/AspNetCore/AspNetCoreOpenApiEndpoints.cs:53

  • Shorthand [] is not a valid initializer for the Security collection. Replace it with an explicit constructor such as new List<OpenApiSecurityRequirement>() to match the property type.
                document.Security ??= [];

src/TodoApp/OpenApi/AspNetCore/AspNetCoreOpenApiEndpoints.cs:54

  • The inner [] for the scopes list in the security requirement is invalid syntax. Use new List<string>() (or the appropriate collection type) instead of [].
                document.Security.Add(new() { [reference] = [] });

@martincostello martincostello merged commit 5935d7e into dotnet-vnext Jul 15, 2025
9 checks passed
@martincostello martincostello deleted the dotnet-10.preview.6 branch July 15, 2025 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file .NET Pull requests that update .NET code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants